Skip to content

fix: restore Docker package build by inverting AGENTS.md symlink#245

Merged
toby-coleman merged 2 commits into
mainfrom
copilot/fix-broken-docker-build
Apr 6, 2026
Merged

fix: restore Docker package build by inverting AGENTS.md symlink#245
toby-coleman merged 2 commits into
mainfrom
copilot/fix-broken-docker-build

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 6, 2026 •
edited
Loading

Summary

The Docker image build started failing after plugboard/cli/ai/AGENTS.md became a symlink to examples/AGENTS.md. Packaging now keeps the CLI template as a real file inside the package and symlinks the example copy back to it.

Changes

  • Canonicalize the packaged template

    • Replace plugboard/cli/ai/AGENTS.md with a real file so wheel/sdist builds do not depend on examples/ being present in the build context.

  • Invert the shared-file relationship

    • Change examples/AGENTS.md to symlink to the packaged CLI copy, preserving a single source of truth without breaking packaging.

  • Guard against regression

    • Add a focused CLI test that asserts the packaged AI template path exists and is not a symlink.
before: plugboard/cli/ai/AGENTS.md -> ../../../examples/AGENTS.md
after:  examples/AGENTS.md -> ../plugboard/cli/ai/AGENTS.md

Copilot AI linked an issue Apr 6, 2026 that may be closed by this pull request
bug: Broken docker build #244
Closed
@toby-coleman
Fix AGENTS template packaging for Docker builds
69c2041 
Agent-Logs-Url: https://github.com/plugboard-dev/plugboard/sessions/e8f47246-ca32-440b-9dfd-3a0017a058f6

Co-authored-by: toby-coleman <13170610+toby-coleman@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix broken Docker build caused by symlink issue fix: restore Docker package build by inverting AGENTS.md symlink Apr 6, 2026
Copilot AI requested a review from toby-coleman April 6, 2026 09:18
@toby-coleman toby-coleman marked this pull request as ready for review April 6, 2026 09:22
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 6, 2026

Benchmark comparison for b86bae99 (base) vs 69c20416 (PR)


------------------------------------------------------------------------------------------------------------------- benchmark: 2 tests ------------------------------------------------------------------------------------------------------------------
Name (time in ms)                                                                         Min                 Max                Mean             StdDev              Median                IQR            Outliers     OPS            Rounds  Iterations
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
test_benchmark_process_run (pr/.benchmarks/Linux-CPython-3.12-64bit/0001_pr)         441.9815 (1.0)      452.0849 (1.0)      447.4148 (1.0)       3.6086 (1.0)      447.5341 (1.0)       3.3014 (1.0)           2;0  2.2351 (1.0)           5           1
test_benchmark_process_run (main/.benchmarks/Linux-CPython-3.12-64bit/0001_base)     451.1273 (1.02)     486.9381 (1.08)     460.5161 (1.03)     15.1397 (4.20)     453.2187 (1.01)     14.7553 (4.47)          1;0  2.1715 (0.97)          5           1
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Legend:
  Outliers: 1 Standard Deviation from Mean; 1.5 IQR (InterQuartile Range) from 1st Quartile and 3rd Quartile.
  OPS: Operations Per Second, computed as 1 / Mean

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@toby-coleman toby-coleman merged commit 68f1077 into main Apr 6, 2026
23 checks passed
@toby-coleman toby-coleman deleted the copilot/fix-broken-docker-build branch April 6, 2026 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@toby-coleman toby-coleman toby-coleman approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@toby-coleman toby-coleman

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

bug: Broken docker build

2 participants

@toby-coleman